-
Notifications
You must be signed in to change notification settings - Fork 4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix typo in values.yaml example for extraVolumeMounts #3960
Conversation
Welcome @kixa! |
/assign @gjtempleton |
@gjtempleton Happy to bump the helm chart version if you want, but since this is pretty much a documentation change, I didn't think it would need it. |
Hey, thanks for the PR, good catch. As this is just a fix in an example I'm happy to bundle this in with another change ( #3954 ) and let the version bump in that take care of publishing a release to include both docs improvements. EDIT: Famous last words, the failing Github action is blocking, so it won't be able to be merged without a version bump, unfortunately. /lgtm |
Thanks! |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: gjtempleton, Kixa The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Incredibly minor, but there's a typo in the Charts default values; the field for volume mounts when mounted read only is
readOnly
, notreadonly
, so if you directly uncomment it,helm install
will fail. This fixes that.